Skip to content

ClickUp Integration#57

Open
unklCharlos wants to merge 18 commits intomainfrom
clickup-integration
Open

ClickUp Integration#57
unklCharlos wants to merge 18 commits intomainfrom
clickup-integration

Conversation

@unklCharlos
Copy link

@unklCharlos unklCharlos commented Apr 29, 2024 •
edited
Loading

  • Sorts out curl requests to create ticket, extract task id and use task id to update requester custom field.
  • Environment variables should already be configured on Circle.
  • Generically renamed ticket template.
  • removes zd creation

Note:

  • Moved some values to become Circle environment variables.
  • May need to sort out OAuth flow; currently just using a personal token.
  • There is a tad bit more cleanup to do as well. I also noticed some different approaches to how I can handle things like extracting the task id from the response using jq. I didn't know if jq would be available until I saw it in the code elsewhere. This did not work for me so I reverted to what I had working.

@unklCharlos unklCharlos self-assigned this Apr 29, 2024
@unklCharlos
Copy link
Author

unklCharlos commented May 17, 2024 •
edited
Loading

@rigoucr @edwtorba @generalredneck I could use some help here.

  1. The CircleCi build is no longer working for this PR. Seems to be related to the public key? I’m a doofus!! I had local db settings in the repo. That is what was making the build fail :awesome:
  2. I was having trouble with the OAuth flow and just set up a personal token for Bender for now. I may need someone else to look at this to sort out. Or at least provide some direction.

@unklCharlos unklCharlos requested a review from rigoucr May 20, 2024 13:08
generalredneck
generalredneck approved these changes Sep 6, 2024
View reviewed changes
Copy link
Contributor

@generalredneck generalredneck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean this looks pretty solid from a code perspective. I say we start giving it a shot and see where it fails.

@unklCharlos
Copy link
Author

unklCharlos commented Sep 20, 2024

@rigoucr Allan approved the code; I also went ahead and removed the ZD ticket creation.

@dalin-
Copy link
Contributor

dalin- commented Jan 22, 2025

Hi all, If I follow, it looks like this can be merged in and released???

@unklCharlos
Copy link
Author

unklCharlos commented Jan 27, 2025

@dalin- I will defer to @rigoucr I recall there was something he was trying to sort out before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rigoucr rigoucr Awaiting requested review from rigoucr

1 more reviewer

@generalredneck generalredneck generalredneck approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@unklCharlos unklCharlos

Labels

Passed code review Ready to merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@unklCharlos @dalin- @generalredneck